Skip to content

feat: add session label controller for Workbench session pods#217

Merged
amdove merged 10 commits intomainfrom
labels-controller
Apr 15, 2026
Merged

feat: add session label controller for Workbench session pods#217
amdove merged 10 commits intomainfrom
labels-controller

Conversation

@amdove
Copy link
Copy Markdown
Contributor

@amdove amdove commented Apr 8, 2026

Description

PTD support for the session label injection feature. See posit-dev/team-operator#123 for the full solution description.

Changes in this repo:

  • team_operator.py: auto-enables the controller in Helm when any site's site.yaml has spec.workbench.sessionLabels configured — no manual flag needed
  • aws_workload_helm.py / azure_workload_helm.py: adds user-group-* to the kube-state-metrics metricLabelsAllowlist so injected labels are exposed as Prometheus metrics
  • docs/CONFIGURATION.md: sessionLabels schema reference
  • docs/team-operator/README.md: usage guide
  • Justfile: fix symlink-binaries to skip binaries already in .local/bin (prevents circular symlinks); add codesign step after CLI build for macOS

Category of change

  • New feature (non-breaking change which adds functionality)

Checklist

  • I have reviewed my own diff and added inline comments on lines I want reviewers to focus on or that I am uncertain about

@amdove amdove marked this pull request as ready for review April 14, 2026 20:29
@amdove amdove requested a review from a team as a code owner April 14, 2026 20:29
@amdove amdove requested a review from stevenolen April 14, 2026 20:30
stevenolen
stevenolen previously approved these changes Apr 14, 2026
Comment thread docs/team-operator/README.md
Comment thread Justfile
@amdove amdove merged commit 293251a into main Apr 15, 2026
4 checks passed
@amdove amdove deleted the labels-controller branch April 15, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants